Skip to content

Change CLI CI timeouts to 40min#14044

Closed
marlenecota wants to merge 1 commit into
microsoft:mainfrom
marlenecota:cli-timeout
Closed

Change CLI CI timeouts to 40min#14044
marlenecota wants to merge 1 commit into
microsoft:mainfrom
marlenecota:cli-timeout

Conversation

@marlenecota

@marlenecota marlenecota commented Oct 30, 2024

Copy link
Copy Markdown
Contributor

Description

When the CLI CI succeeds, it usually only takes < 20 min. Sometimes it goes up to low 30s, so reducing the timeout from 120 to 40 to increase speed of re-runs.

Changelog

Should this change be included in the release notes: no

Microsoft Reviewers: Open in CodeFlow

@marlenecota marlenecota requested review from a team as code owners October 30, 2024 20:45
@marlenecota

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Contributor
Azure Pipelines successfully started running 1 pipeline(s), but failed to run 1 pipeline(s).

@danielayala94 danielayala94 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@Yajur-Grover Yajur-Grover left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The 'Verify CliInitWindows PaperX64ReleaseCsChakra' check on Jon's PR here is still running at ~40 minutes - is this not an issue?

edit: looks like it timed out, and when it doesn't, it takes less time to run.👍

@marlenecota

Copy link
Copy Markdown
Contributor Author

The 'Verify CliInitWindows PaperX64ReleaseCsChakra' check on Jon's PR here is still running at ~40 minutes - is this not an issue?

That's that I am trying to mitigate. When the CLI checks succeed, they usually do so in less than 20 minutes. So instead of waiting 2 hours for the check to timeout, we can bail at 40 and re-run.

@marlenecota marlenecota reopened this Oct 30, 2024
@marlenecota marlenecota enabled auto-merge (squash) October 30, 2024 20:59
@marlenecota

Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Contributor
Azure Pipelines successfully started running 1 pipeline(s), but failed to run 1 pipeline(s).

@marlenecota

Copy link
Copy Markdown
Contributor Author

This PR got in a weird state with CI. Replacing with #14045

auto-merge was automatically disabled October 30, 2024 21:13

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants